Skip to content

Conversation

@hrfarmer
Copy link
Collaborator

@hrfarmer hrfarmer commented Oct 3, 2025

Automated Integration Tests

Description

This PR adds multiple different files/changes that, in all, enable full automated integration tests on each commit.

  • Adds an integration job to ci.yaml that runs on a self hosted runner with a flight controller board connected
  • Adds a bootloader target to the Makefile that restart a flight controller board into bootloader mode, and adds a gds-integration command that disables the gds website
  • Adds a script that triggers the bootloader reset command on Fprime
  • Adds a pytest fixture to the integration test suite that automatically starts the gds before running tests, and closing it once they're done
    • Means the gds would need to be closed before tests are ran

How Has This Been Tested?

Integration tests have been ran on the board and work correctly
The self-hosted runner with the integration job has been tested to work correctly locally, although still need to write a proper set-up guide to get a self-hosted runner completely ready

  • Unit tests
  • Integration tests
  • Z Tests
  • Manual testing (describe steps)

Screenshots / Recordings (if applicable)

image

Action Link

Checklist

  • Written detailed sdd with requirements, channels, ports, commands, telemetry defined and correctly formatted and spelled
  • Have written relevant integration tests and have documented them in the sdd
  • Have done a code review with
  • Have tested this PR on every supported board with correct board definitions

Further Notes / Considerations

@hrfarmer hrfarmer requested a review from nateinaction October 3, 2025 22:31
@hrfarmer hrfarmer marked this pull request as draft October 4, 2025 16:03
@hrfarmer hrfarmer changed the title Bootloader Python Script Automated Integration Tests Oct 5, 2025
@hrfarmer hrfarmer marked this pull request as ready for review October 7, 2025 22:45
Copy link
Contributor

@Mikefly123 Mikefly123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! So cool!

@hrfarmer hrfarmer merged commit 4bbe9bc into main Oct 7, 2025
22 of 27 checks passed
@hrfarmer hrfarmer deleted the bootloader-script branch October 7, 2025 22:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants